Replace aAsConst with std::as_const in tests
authorClaudio Cambra <claudio.cambra@nextcloud.com>
Thu, 5 Dec 2024 03:38:12 +0000 (11:38 +0800)
committerMatthieu Gallien <matthieu.gallien@nextcloud.com>
Fri, 20 Dec 2024 15:26:57 +0000 (16:26 +0100)
Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
test/syncenginetestutils.cpp
test/testsyncconflict.cpp
test/testsyncengine.cpp
test/testsyncjournaldb.cpp
test/testsyncmove.cpp
test/testsyncvirtualfiles.cpp

index d3f351f1c725d60b190afcd429312f740f50adb0..30c6254e5bfb6dcd7722dbd6b7f8b3e56821e967 100644 (file)
@@ -585,7 +585,7 @@ void FakePutMultiFileReply::respond()
         totalSize += fileInfo->size;
     });
 
-    for(auto fileInfo : qAsConst(_allFileInfo)) {
+    for(auto fileInfo : std::as_const(_allFileInfo)) {
         QJsonObject fileInfoReply;
         fileInfoReply.insert("error", QStringLiteral("false"));
         fileInfoReply.insert("etag", QLatin1String{fileInfo->etag});
@@ -1045,12 +1045,12 @@ QJsonObject FakeQNAM::forEachReplyPart(QIODevice *outgoingData,
     const QString boundaryValue = QStringLiteral("--") + contentType.mid(boundaryPosition, contentType.length() - boundaryPosition - 1) + QStringLiteral("\r\n");
     auto stringPutPayloadRef = QString{stringPutPayload}.left(stringPutPayload.size() - 2 - boundaryValue.size());
     auto allParts = stringPutPayloadRef.split(boundaryValue, Qt::SkipEmptyParts);
-    for (const auto &onePart : qAsConst(allParts)) {
+    for (const auto &onePart : std::as_const(allParts)) {
         auto headerEndPosition = onePart.indexOf(QStringLiteral("\r\n\r\n"));
         auto onePartHeaderPart = onePart.left(headerEndPosition);
         auto onePartHeaders = onePartHeaderPart.split(QStringLiteral("\r\n"));
         QMap<QString, QByteArray> allHeaders;
-        for(const auto &oneHeader : qAsConst(onePartHeaders)) {
+        for(const auto &oneHeader : std::as_const(onePartHeaders)) {
             auto headerParts = oneHeader.split(QStringLiteral(": "));
             allHeaders[headerParts.at(0).toLower()] = headerParts.at(1).toLatin1();
         }
index ce358feeddb666aaab4520e8deceef43da565623..1db24cf24e7cc9176f3b4ad2d0b101b5f32da35d 100644 (file)
@@ -45,7 +45,7 @@ bool expectAndWipeConflict(FileModifier &local, FileInfo state, const QString pa
     auto base = state.find(pathComponents.parentDirComponents());
     if (!base)
         return false;
-    for (const auto &item : qAsConst(base->children)) {
+    for (const auto &item : std::as_const(base->children)) {
         if (item.name.startsWith(pathComponents.fileName()) && item.name.contains("(conflicted copy")) {
             local.remove(item.path());
             return true;
index c745ddcad4ffed1c3d43397584513d27a4d81aa3..079dd18c1f943ec32c5236467bc906e86d8acc7d 100644 (file)
@@ -41,7 +41,7 @@ bool expectConflict(FileInfo state, const QString path)
     auto base = state.find(pathComponents.parentDirComponents());
     if (!base)
         return false;
-    for (const auto &item : qAsConst(base->children)) {
+    for (const auto &item : std::as_const(base->children)) {
         if (item.name.startsWith(pathComponents.fileName()) && item.name.contains("(case clash from")) {
             return true;
         }
index 426b5bfd836d619353544da04c7371cb3d30e74c..d5897bdf75627226ec877a7f977a4a367126fa46 100644 (file)
@@ -293,13 +293,13 @@ private slots:
             << "foo bla bar/file"
             << "fo_"
             << "fo_/file";
-        for (const auto& elem : qAsConst(elements)) {
+        for (const auto& elem : std::as_const(elements)) {
             makeEntry(elem);
         }
 
         auto checkElements = [&]() {
             bool ok = true;
-            for (const auto& elem : qAsConst(elements)) {
+            for (const auto& elem : std::as_const(elements)) {
                 SyncJournalFileRecord record;
                 if (!_db.getFileRecord(elem, &record) || !record.isValid()) {
                     qWarning() << "Missing record: " << elem;
index 1475013cf3226f934f2832ea3904c81edeb7f0ac..8b5ef5806aae3b4528b5213eb8115459c252dd3f 100644 (file)
@@ -77,7 +77,7 @@ bool expectAndWipeConflict(FileModifier &local, FileInfo state, const QString pa
     auto base = state.find(pathComponents.parentDirComponents());
     if (!base)
         return false;
-    for (const auto &item : qAsConst(base->children)) {
+    for (const auto &item : std::as_const(base->children)) {
         if (item.name.startsWith(pathComponents.fileName()) && item.name.contains("(conflicted copy")) {
             local.remove(item.path());
             return true;
@@ -669,7 +669,7 @@ private slots:
         }
         conflicts = findConflicts(currentLocal.children["B4"]);
         QCOMPARE(conflicts.size(), 1);
-        for (const auto& c : qAsConst(conflicts)) {
+        for (const auto& c : std::as_const(conflicts)) {
             QCOMPARE(currentLocal.find(c)->contentChar, 'L');
             local.remove(c);
         }
index c50d471ce44cc63fdb617cb54ce6c045baa48f0a..47dabce84f2a2da2f62169a7e2a1468f9c2fbcdc 100644 (file)
@@ -32,7 +32,7 @@ bool expectConflict(FileInfo state, const QString path)
     auto base = state.find(pathComponents.parentDirComponents());
     if (!base)
         return false;
-    for (const auto &item : qAsConst(base->children)) {
+    for (const auto &item : std::as_const(base->children)) {
         if (item.name.startsWith(pathComponents.fileName()) && item.name.contains("(case clash from")) {
             return true;
         }